Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adding new document for contribution handling #8761

Merged
merged 12 commits into from
Sep 25, 2023
Merged

Adding new document for contribution handling #8761

merged 12 commits into from
Sep 25, 2023

Conversation

duncanmacmichael
Copy link
Member

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

@microsoft-github-policy-service microsoft-github-policy-service bot added the needs-triage Issue needs to be triaged by the area owners label Aug 19, 2023
@duncanmacmichael
Copy link
Member Author

@bpulliam Fixed the links after the build validation failed - can you re-run? I think it only runs upon opening the PR.

@bpulliam bpulliam added documentation An issue with existing documentation or a request for documenation of a new topic and removed needs-triage Issue needs to be triaged by the area owners labels Aug 21, 2023
@ranjeshj
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ojhad ojhad mentioned this pull request Aug 23, 2023
Closed
@ojhad
Copy link
Contributor

ojhad commented Aug 23, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@evelynwu-msft
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ranjeshj ranjeshj mentioned this pull request Aug 29, 2023
Closed
dotMorten
dotMorten suggested changes Aug 29, 2023
View reviewed changes
docs/contribution_handling.md Outdated Show resolved Hide resolved
docs/contribution_handling.md Outdated Show resolved Hide resolved
docs/contribution_handling.md Outdated Show resolved Hide resolved
riverar
riverar reviewed Aug 29, 2023
View reviewed changes
docs/contribution_handling.md Outdated Show resolved Hide resolved
docs/contribution_handling.md Outdated Show resolved Hide resolved
docs/contribution_handling.md Outdated Show resolved Hide resolved
@dotMorten
Copy link
Contributor

I think bottom line, this PR seems to go directly against the Windows App SDK Mission Statement

sylveon
sylveon reviewed Aug 30, 2023
View reviewed changes
docs/contribution_handling.md Show resolved Hide resolved
docs/contribution_handling.md Outdated Show resolved Hide resolved
@duncanmacmichael
Copy link
Member Author

Thank you for your feedback, everyone. If I didn't reply to your comment yet (as of the end of day on 08/30/23), then that means we're discussing it internally. All of this is very valuable feedback as we shape this messaging!

@duncanmacmichael
Copy link
Member Author

@bpulliam or @codendone , please merge this PR at your earliest convenience, thanks!

dotMorten
dotMorten reviewed Sep 17, 2023
View reviewed changes
docs/contribution_handling.md

### Feature Planning

Our long term vision for WinUI is to be the best UX/UI native framework for Windows PCs that provides premium visuals, motion, accessibility, usability, and support for all input modalities.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding what is somewhat of a vision statement. It's nice to know what drives your overall priorities, albeit I'm a bit sad to see developer experience isn't part of it. It's all imho a bit moot if we give up half-way to getting there.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would be very much appreciated if the roadmap for developers can be defined, too. At present, the information about native Windows development is confusing, to put it mildly. The various options are lacking their properly defined places, and WinUI 3 is not making is better, especially when the current state of the SDK and the development tools feel more like a beta than a production-ready product.

@soroshsabz soroshsabz mentioned this pull request Sep 21, 2023
Closed
@codendone
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codendone codendone merged commit 4c50e61 into microsoft:main Sep 25, 2023
2 checks passed
@riverar riverar mentioned this pull request Nov 16, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nikolayvpavlov nikolayvpavlov nikolayvpavlov left review comments

@Scottj1s Scottj1s Scottj1s left review comments

@Sergio0694 Sergio0694 Sergio0694 left review comments

@ojhad ojhad ojhad left review comments

@sylveon sylveon sylveon left review comments

@dotMorten dotMorten dotMorten requested changes

@hartez hartez hartez left review comments

@riverar riverar riverar left review comments

@chigy chigy chigy approved these changes

Assignees
No one assigned
Labels
documentation An issue with existing documentation or a request for documenation of a new topic
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.